Skip to content

feat: add /gsd:review cross-AI peer review command#925

Open
ashanuoc wants to merge 1 commit intogsd-build:mainfrom
ashanuoc:feat/cross-ai-review
Open

feat: add /gsd:review cross-AI peer review command#925
ashanuoc wants to merge 1 commit intogsd-build:mainfrom
ashanuoc:feat/cross-ai-review

Conversation

@ashanuoc
Copy link

@ashanuoc ashanuoc commented Mar 3, 2026

Summary

  • Add /gsd:review command that invokes external AI CLIs (Gemini, Claude, Codex) to independently review phase plans
  • Produces structured REVIEWS.md with per-reviewer feedback sections (YAML frontmatter + Summary/Strengths/Concerns/Suggestions/Risk Assessment)
  • Add --reviews flag to /gsd:plan-phase to incorporate review feedback into planning
  • 11 new tests, all 549 tests passing

Test plan

  • npm test — all 549 tests pass (0 failures)
  • review check-cli returns JSON with CLI availability
  • review build-prompt --phase N creates prompt file
  • review write-reviews --phase N writes REVIEWS.md
  • init review N returns phase info with cli_available
  • init plan-phase N includes has_reviews when REVIEWS.md exists

Add a new command that invokes external AI CLIs (Gemini, Claude, Codex)
to independently review phase plans and produces a REVIEWS.md document
that can be fed back into planning via --reviews flag.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ashanuoc ashanuoc requested a review from glittercowboy as a code owner March 3, 2026 22:49
@azeitler-dotcom
Copy link

I like this idea. In fact, I've been doing something manually that I figured I should try to automate (or try to see if I could get added as a feature directly to gsd). My approach (which is not elegant for sure) is to copy my codebase right after I built my CONTEXT.MD file for a phase. I then have a separate AI (like in Codex or using GLM through Claude Code) to create the RESEARCH.MD and PLAN.MD file. I then have the original directory with Claude Code + Anthropic models also do an independent planning and research. I then have copied the GLM and/or Codex PLAN.md (sometimes I've done more than 1 alternative) into an "/externals" folder I have in my codebase. I tell my original Claude Code to compare the plan and research files and see if there is anything it can use that is done better in that version. It usually finds at least one thing.

I've been doing this for a few phases and it helps fill things out more. Your approach is way more streamlined. Part of what I was trying to achieve was independent producers of the files, sort of like having two people look at the same directions and see if they come to different conclusions.

Oh, and I also more recently even tried doing the split again, where I took the merged PLAN.md files (with input from all runs merged) and then let two independent directories execute on the same plan. Then I made a custom skill that produces a diff. I then feed the diff to my original instance of Claude Code so it can compare the changes it made to the one the other model made when executing the same PLAN.md. This also allowed for some better execution ideas to flow back into the original instance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants